Skip to content

test(forge): live-oracle GitHub legs authenticate as Apps, not PATs (RIG-3096) - #837

Draft
rigel-mintaka wants to merge 2 commits into
mainfrom
compass-forge/rig-3096-live-oracle-apps
Draft

test(forge): live-oracle GitHub legs authenticate as Apps, not PATs (RIG-3096)#837
rigel-mintaka wants to merge 2 commits into
mainfrom
compass-forge/rig-3096-live-oracle-apps

Conversation

@rigel-mintaka

Copy link
Copy Markdown
Contributor

Swap the forge live-contract oracle's author + reviewer identities from
fine-grained bot PATs to GitHub App installation-token sources, exercising
the production mint path (RS256 App JWT -> installation access token) the
deployed server uses after the RIG-3090 credential cutover.

The Linear legs stay on the app-actor token (client_credentials mint); the
retained-Linear-user-cred plan from the frozen record is dropped per Matt
(agent delegation only, moving off the user model).

Refs RIG-3096
Co-authored-by: Matt Wilkinson matt@rigel.build

…RIG-3096)

Swap the forge live-contract oracle's author + reviewer identities from
fine-grained bot PATs to GitHub App installation-token sources, exercising
the production mint path (RS256 App JWT -> installation access token) the
deployed server uses after the RIG-3090 credential cutover.

The Linear legs stay on the app-actor token (client_credentials mint); the
retained-Linear-user-cred plan from the frozen record is dropped per Matt
(agent delegation only, moving off the user model).

Refs RIG-3096
Co-authored-by: Matt Wilkinson <matt@rigel.build>
@linear-code

linear-code Bot commented Sep 2, 2026

Copy link
Copy Markdown

RIG-3096

@trunk-io

trunk-io Bot commented Sep 2, 2026

Copy link
Copy Markdown

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

Compass engineering docs preview: https://compass-forge-rig-3096-live.compass-eng-docs.pages.dev

Deployed from compass-forge/rig-3096-live-oracle-apps at fd83ead.

…docs (RIG-3096)

Review low-severity doc-precision fixes on PR #837:
- Note requireLive eagerly builds BOTH App identities regardless of which
  the caller uses, so a reviewer-only misconfig still fails an author-only
  test loud (the intended all-creds-or-skip gate).
- Clarify a malformed PEM is caught at first mint on a live leg (lazy key
  resolution), not at liveAppSource setup \u2014 still red, never silent-green.

Comment-only; no behavior change.

Refs RIG-3096
Co-authored-by: Matt Wilkinson <matt@rigel.build>
@rigel-mintaka

Copy link
Copy Markdown
Contributor Author

Status: review-clean; held draft on purpose; unblock is RIG-3151.

Swaps the forge live-oracle's GitHub author+reviewer identities from bot PATs to GitHub App installation-token sources (RIG-3096, frozen design section T5). Linear legs stay app-actor (no user credential, per the pivot off the user model).

Why the forge check is red and that's expected: the forge-oracle assert-ran guard (ci.yml:1071-1124) turns a credential-unset t.Skip into a hard red, so a required check never passes vacuously. The six App Actions secrets do not exist yet, so the oracle legs skip and the guard reds. Every other check is green (e2e, pgtest, microvm, all moon lanes, deploy).

Unblock chain:

  1. RIG-3151 (human-action, Matt): register the two compass-forge-testbed Apps (author + reviewer) and stage the six App-cred values into the orion orion-platform-secrets stack.
  2. Orion IaC PR (compass-forge lane): create the six LIVEGITHUB_*_APP_* Actions secrets on this repo, retire the two PAT rows.
  3. forge goes green, then this PR is promoted out of draft.

Held draft precisely so it cannot land before step 2: landing with the secrets unset would red main's forge gate for every forge-touching PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant